Skip to content

Conversation

@sylwia-budzynska
Copy link
Contributor

It looks like the paths in the example were switched around. @anticomputer is this correct?

@Copilot Copilot AI review requested due to automatic review settings October 9, 2025 19:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates environment variable paths in README documentation to correct inconsistent path mappings between the example configuration and deployment command.

  • Standardizes CODEQL_DBS_BASE_PATH to use "/codeql_databases" in the environment configuration example
  • Updates the deployment example to use consistent path mappings between MY_DATA and CODEQL_DBS_BASE_PATH

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@m-y-mo
Copy link
Contributor

m-y-mo commented Oct 10, 2025

It does look like it, but then in the second example, /codeql_databases is not mounted, so maybe it makes more sense to change both the CODEQL_DBS_BASE_PATH to /app/my_data/codeql_databases instead so at least it'd be mounted in the docker container in the examples?

m-y-mo and others added 2 commits October 23, 2025 13:53
Co-authored-by: Sylwia Budzynska <102833689+sylwia-budzynska@users.noreply.github.com>
Co-authored-by: Sylwia Budzynska <102833689+sylwia-budzynska@users.noreply.github.com>
@m-y-mo m-y-mo merged commit 29dbed4 into main Oct 23, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants